Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

SingleGeneAiExpressionReporter #255

Open
wants to merge 43 commits into
base: master
Choose a base branch
from
Open

SingleGeneAiExpressionReporter #255

wants to merge 43 commits into from

Conversation

bobular
Copy link
Member

@bobular bobular commented Feb 12, 2025 •
edited
Loading

WIP.

Current status:

  1. it compiles!
  2. the AI, WDK table retrieval and Reporter logic are all wired together (hopefully)
  3. but it hasn't been run yet...
  4. no caching yet
  5. it still expects the API key in env

bobular and others added 30 commits February 12, 2025 22:03
@bobular
WIP
8d16493
@bobular
it compiles - at least
f85f2a0
@bobular
all wired together and compiles
089511c
@bobular
extra comment
5752cc4
@bobular
one more comment
4ce3f5f
@bobular
move CacheMode into separate file
eb203df
@ryanrdoherty
Merge branch 'master' into ai-expression
9ce38a4
@bobular
WIP cache wiring
dfab346
@bobular
more cache wrangling
813d2f1
@ryanrdoherty
a few changes for error handling and to set up caching
5c323c3
@ryanrdoherty
Merge branch 'ai-expression' into ai-expression-rrd
e53e86e
@ryanrdoherty
Massage class roles
0e6b945
@ryanrdoherty
Merge branch 'master' into ai-expression
135881e
@ryanrdoherty
Merge branch 'ai-expression' into ai-expression-rrd
8e7b3aa
@ryanrdoherty
Checkpoint commit; finished up AiExpressionCache and just need to tri…
86dad07 
…m down Summarizer
@ryanrdoherty
Clean cache logic out of summarizer
be6a61a
@ryanrdoherty
Merge branch 'master' into ai-expression
afeb850
@ryanrdoherty
Merge branch 'ai-expression' into ai-expression-rrd
13d47e6
@bobular
make cache validation digest symmetrical for both levels of AI query
32a7351
@bobular
improved code formatting of JSONSchema definitions
44b65aa
@ryanrdoherty
Incorporate AI chat model string into result digests; they will becom…
4bcb86f 
…e out-of-date if the chat model changes
@ryanrdoherty
Reparallelize experiment lookups
be79018
@bobular
increase timeout and use ChatModel.toString() to fix exception
18f0870
@bobular
getGeneId() fix
a579969
@bobular
bugfix
6693be5
@bobular
reworked summary prompt to avoid generalities and for clarity
4f52d37
@bobular
prompt for structured summary paragraph
6a5a078
@ryanrdoherty
Merge branch 'master' into ai-expression
d60cb84
@ryanrdoherty
Merge branch 'ai-expression' into ai-expression-rrd
7dd234a
@ryanrdoherty
Merge pull request #256 from VEuPathDB/ai-expression-rrd
f2e7845 
Temp draft PR for easier diffing
@bobular bobular marked this pull request as ready for review March 2, 2025 16:45
bobular added 2 commits March 2, 2025 20:15
@bobular
Merge pull request #261 from VEuPathDB/ai-expression-topics
b378524 
Rename "sections" as "topics", change prompting and attempt to sort by `biological_importance`
@bobular
Merge remote-tracking branch 'origin/master' into ai-expression
70d42d5
bobular
bobular commented Mar 4, 2025
View reviewed changes
Model/src/main/java/org/apidb/apicommon/model/report/ai/expression/Summarizer.java
.putAdditionalProperty("required", JsonValue.from(List.of(
"headline",
"one_paragraph_summary",
"dataset_ids"
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This dataset_ids should not be here. I will remove it before we merge!

Copy link
Member Author

@bobular bobular Mar 4, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should be topics

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bobular @ryanrdoherty